Skip to content

fix(eng-10262): added correct encoding of next query parameter#883

Closed
ihorsokhanexoft wants to merge 10 commits intoCenterForOpenScience:feature/osf4i-in-progress-ssofrom
ihorsokhanexoft:fix/ENG-10262
Closed

fix(eng-10262): added correct encoding of next query parameter#883
ihorsokhanexoft wants to merge 10 commits intoCenterForOpenScience:feature/osf4i-in-progress-ssofrom
ihorsokhanexoft:fix/ENG-10262

Conversation

@ihorsokhanexoft
Copy link

@ihorsokhanexoft ihorsokhanexoft commented Feb 13, 2026

https://openscience.atlassian.net/browse/ENG-10262

Purpose

For LOCAL env only:

A new next query parameter must be the first one to be encrypted and then encrypted one more time with service query parameter together.
So everything after localhost:8080/login?service= is encrypted and the next query param with its value is encrypted twice.

Also in the middle of the url, we should use ? instead of & (%3F - encoded, between login and next words) so that the next parameter belongs to service, not the main request

http://192.168.168.167:8080/login?service=http%3A%2F%2Flocalhost%3A5000%2Flogin%3Fnext%3Dhttp%253A%252F%252Flocalhost%253A4200%252F

And CAS should handle login, not angular, so we replace 4200 port in service query parameter by 5000

nsemets and others added 9 commits January 30, 2026 11:24
- Ticket: [ENG-10048]
- Feature flag: n/a

## Summary of Changes
1. Added registry info to overview and metadata pages.
)

- Ticket: https://openscience.atlassian.net/browse/ENG-10047
- Feature flag: n/a

## Purpose

User profile pages do not currently display a user’s affiliated institution(s), even when the user has active institutional affiliations set in OSF. This makes it difficult for others to understand a user’s institutional context and reduces the visibility of institutional participation on the platform.


## Summary of Changes

Implement affiliated Institution(s) on User Profile Page showing
…a default license in admin. (#796)

- Ticket: https://openscience.atlassian.net/browse/ENG-9042
- Feature flag: n/a

## Purpose
Each registries, preprints, and collections provider sets a default license in admin. 

## Summary of Changes
These should be preselected on all registration drafts on that provider, and the user can change them from there.

All provider types need a serialized default license.
…ion cards (#855)

- Ticket: https://openscience.atlassian.net/browse/ENG-6719
- Feature flag: n/a

## Purpose
Some registries members use funder information to determine priority or relevance of submissions.  They currently must drill down several layers within their moderation workflow to find this information.
## Purpose
The logged-in user was not redirected from the home page to the dashboard because the required authentication data was missing.

## Summary of Changes
1. Removed home route from `app.server.route.ts`.
…ults #870

- Ticket: [ENG-10148]
- Feature flag: n/a

## Summary of Changes
1. Added loading message.
- Ticket: [ENG-10251]
- Feature flag: n/a

## Summary of Changes
1. Renamed all instances of `.models.ts` to `.model.ts`.
2. Updated all internal references and import paths in components, services, and pipes to reflect the new file names.
3. Updated Jest unit tests to ensure imports within `.spec.ts` files are pointing to the corrected model paths.
4. Verified that no duplicate model definitions exist under the old naming scheme.
5. Remove animations.
…t overview and institutions (#877)

- Ticket: [ENG-10252]
- Feature flag: n/a

## Summary of Changes
1. Added unit tests for project overview and institutions.
nsemets
nsemets previously approved these changes Feb 16, 2026
@nsemets
Copy link
Collaborator

nsemets commented Feb 17, 2026

@ihorsokhanexoft Fix lint issue.

@ihorsokhanexoft ihorsokhanexoft changed the base branch from feature/pbs-26-2 to feature/osf4i-in-progress-sso February 17, 2026 11:35
@ihorsokhanexoft ihorsokhanexoft force-pushed the fix/ENG-10262 branch 2 times, most recently from 6093b43 to 4205679 Compare February 17, 2026 11:42
@ihorsokhanexoft ihorsokhanexoft closed this by deleting the head repository Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants